Skip to content

ci: publish Docker image under repo org#199

Merged
peaktwilight merged 1 commit intomainfrom
fix/docker-publish-ghcr-org
May 7, 2026
Merged

ci: publish Docker image under repo org#199
peaktwilight merged 1 commit intomainfrom
fix/docker-publish-ghcr-org

Conversation

@Darkroom4364
Copy link
Copy Markdown
Collaborator

@Darkroom4364 Darkroom4364 commented May 5, 2026 •
edited by coderabbitai Bot
Loading

Summary

  • publish the Docker image to the repository-owner GHCR namespace instead of the stale personal namespace
  • authenticate GHCR pushes with the default GITHUB_TOKEN
  • update Docker image references in README and docs to ghcr.io/pwnkit-labs/pwnkit:latest

Closes #174.

Verification

  • git diff --cached --check
  • ruby -e "require 'yaml'; YAML.load_file('.github/workflows/docker-publish.yml'); puts 'yaml ok'"

Note: I did not run workflow_dispatch locally; that still needs to be verified by GitHub Actions after merge or from this branch.

Summary by CodeRabbit

  • Chores

    • Updated Docker publishing workflow to use native GitHub Actions authentication instead of personal credentials.
    • Standardized Docker image registry path across configuration.

  • Documentation

    • Updated Docker image references and corrected naming consistency in Quick Start, configuration, and getting-started guides.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 5, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 7275365a-6057-4314-8968-f04406675cf4

📥 Commits

Reviewing files that changed from the base of the PR and between 9232e08 and d3a4961.

📒 Files selected for processing (4)
  • .github/workflows/docker-publish.yml
  • README.md
  • docs/src/content/docs/configuration.md
  • docs/src/content/docs/getting-started.md
📝 Walkthrough

Walkthrough

The pull request migrates Docker image publishing from the personal peaktwilight namespace to the organization-owned pwnkit-labs namespace. It updates the GitHub Actions workflow to derive the image owner dynamically from github.repository_owner, switches to GHCR authentication via github.actor and GITHUB_TOKEN, and harmonizes all documentation and example references to use the correct lowercase GHCR path.

Changes

Docker Image Namespace & GHCR Authentication Migration

Layer / File(s) Summary
Workflow Authentication & Resolution
.github/workflows/docker-publish.yml		 GHCR login switches from hardcoded peaktwilight user + GHCR_PEAKTWILIGHT_TOKEN to github.actor + secrets.GITHUB_TOKEN. New "Resolve image name" step computes ghcr.io/${OWNER}/pwnkit (lowercased) from github.repository_owner, resolving to ghcr.io/pwnkit-labs/pwnkit.
Metadata & Labels
.github/workflows/docker-publish.yml		 Metadata action now uses dynamically resolved image name and org.opencontainers.image.source label updated to https://github.com/PwnKit-Labs/pwnkit.
Documentation & Examples
README.md, docs/src/content/docs/configuration.md, docs/src/content/docs/getting-started.md		 Image references updated from ghcr.io/peaktwilight/pwnkit:latest or ghcr.io/PwnKit-Labs/pwnkit:latest (mixed casing) to consistent ghcr.io/pwnkit-labs/pwnkit:latest.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • PwnKit-Labs/pwnkit#184: Modifies the same Docker publish workflow; check for potential conflicts or superseded changes regarding GHCR authentication and image namespace.

Poem

🐰 A Docker hop from old to new,
From personal to org—the right thing to do!
pwnkit-labs now owns the name,
With lowercase paths, the workflow's the same.
No frozen digests, just fresh releases—
Configuration fixed in little pieces! 🐇✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely summarizes the main change: publishing Docker images under the repository org namespace instead of a personal namespace.
Linked Issues check ✅ Passed The PR directly addresses all coding requirements from issue #174: workflow resolves owner from github.repository_owner, uses GITHUB_TOKEN for GHCR authentication, updates image references in README and docs.
Out of Scope Changes check ✅ Passed All changes are directly related to migrating Docker image publishing from personal to org namespace and updating downstream references accordingly.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/docker-publish-ghcr-org

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Darkroom4364 Darkroom4364 requested a review from peaktwilight May 6, 2026 08:00
@peaktwilight peaktwilight merged commit 65eeeaa into main May 7, 2026
13 checks passed
@peaktwilight peaktwilight deleted the fix/docker-publish-ghcr-org branch May 7, 2026 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@peaktwilight peaktwilight Awaiting requested review from peaktwilight

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

ci(docker-publish): publish under pwnkit-labs org instead of peaktwilight

2 participants

@Darkroom4364 @peaktwilight